Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IS-1577] Allow veileder to filter on dnr-dates as well #357

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

eirikdahlen
Copy link
Contributor

@eirikdahlen eirikdahlen commented Sep 5, 2023

Vi utvider lista fra 31 til 71 datoer, ettersom at NAV Oppfølging Utland ønsker å filtrere på d-nr i oversikten også. D-nr er som et vanlig fnr, men de to første sifferene har +40 i verdi.

image

Vi utvider lista fra 31 til 71 datoer, ettersom at NAV Oppfølging Utland ønsker å filtrere på d-nr i oversikten også. D-nr er som et vanlig fnr, men de to første sifferene har +40 i verdi.
@eirikdahlen eirikdahlen requested a review from a team as a code owner September 5, 2023 10:00
@andersrognstad
Copy link
Contributor

andersrognstad commented Sep 5, 2023

Vi utvider lista fra 31 til 71 datoer, ettersom at NAV Oppfølging Utland ønsker å filtrere på d-nr i oversikten også. D-nr er som et vanlig fnr, men de to første sifferene har +40 i verdi.

image

Pirk, men blir det rart å kalle det Fødselsdato (i label over feltet) når man åpner for tall som ikke er datoer. Kunne man brukt Fnr/Dnr i stedet f.eks. 💭

@eirikdahlen
Copy link
Contributor Author

Vi utvider lista fra 31 til 71 datoer, ettersom at NAV Oppfølging Utland ønsker å filtrere på d-nr i oversikten også. D-nr er som et vanlig fnr, men de to første sifferene har +40 i verdi.
image

Pirk, men blir det rart å kalle det Fødselsdato (i label over feltet) når man åpner for tall som ikke er datoer. Kunne man brukt Fnr/Dnr i stedet f.eks. 💭

Ah, godt poeng 🤔 Det er jo heller ikke fnr/dnr man velger sånn sett, så evt måtte det blitt noe a la Fnr/dnr som starter på... som overskrift eller noe sånt?
Den kan jo evt gå ut sånn den er nå, og så kunne man endret det dersom folk synes det er rart?

@andersrognstad
Copy link
Contributor

Vi utvider lista fra 31 til 71 datoer, ettersom at NAV Oppfølging Utland ønsker å filtrere på d-nr i oversikten også. D-nr er som et vanlig fnr, men de to første sifferene har +40 i verdi.
image

Pirk, men blir det rart å kalle det Fødselsdato (i label over feltet) når man åpner for tall som ikke er datoer. Kunne man brukt Fnr/Dnr i stedet f.eks. 💭

Ah, godt poeng 🤔 Det er jo heller ikke fnr/dnr man velger sånn sett, så evt måtte det blitt noe a la Fnr/dnr som starter på... som overskrift eller noe sånt? Den kan jo evt gå ut sånn den er nå, og så kunne man endret det dersom folk synes det er rart?

Kan godt gå ut ja, så kanskje ta en liten runde med UX på det etterpå for å finne en passende label.

@eirikdahlen eirikdahlen merged commit 8d9db7b into master Sep 6, 2023
4 checks passed
@eirikdahlen eirikdahlen deleted the 1577-dato-filter branch September 6, 2023 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants